-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Site Multiplicities to Structure Specific Metadata #248
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, thank you. Once the checks pass we'll merge!
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #248 +/- ##
==========================================
+ Coverage 44.94% 47.23% +2.29%
==========================================
Files 15 15
Lines 1266 1268 +2
==========================================
+ Hits 569 599 +30
+ Misses 697 669 -28 ☔ View full report in Codecov by Sentry. |
@xhqu1981 You good with me merging? Looks good to me overall! |
Yes. Please go ahead. Best, Xiaohui
…On Wed, Feb 28, 2024 at 8:22 PM Matthew Carbone ***@***.***> wrote:
@xhqu1981 <https://github.com/xhqu1981> You good with me merging? Looks
good to me overall!
—
Reply to this email directly, view it on GitHub
<#248 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABDONX5C26FHIRMXWXM4QJDYV7J4JAVCNFSM6AAAAABD6HWXWCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZQGIYDQMZQG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@xhqu1981 It looks good to me. Thanks. |
The site multiplicities are added as a dictionary, mapping a site index into its multiplicity, to the metadata.json file in each structure folder.